-
-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes the DT ajax error #312
Conversation
✅ All contributors have signed the CLA |
I have read the CLA Document and I hereby sign the CLA |
Code Coverage Summary
Diff against main
Results for commit: fb9af77 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Require changes in other modules as well but this is excellent work. Will imrpve
Signed-off-by: Marcin <[email protected]>
Hey @vedhav I don't see the plot - this is what we ended up on a live session, before we abandoned the |
@vedhav ignore my previous comment, something different must have happened and blocked the plot/browser. Now, when restarted, I can see the plot |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vedhav I applied changes to other modules :)
Thanks for the help, would you mind having one last look?
Closes #284
Currently, the implementation is only done for the
tm_g_gh_correlationplot
so please test the example for this module alone: